Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Add GitHub actions workflows for unit testing, checking formatting, and updating actions #2

Merged
merged 5 commits into from
Jul 25, 2024
Merged

Add GitHub actions workflows for unit testing, checking formatting, and updating actions #2

merged 5 commits into from
Jul 25, 2024

Conversation

msimberg
Copy link
Collaborator

  • Adds a workflow that builds and runs the unit test in debug and release mode. It enables warnings and -Werror for uenv itself.
  • Adds a workflow that checks formatting with clang-format.
  • Adds a dependabot workflow that checks for actions updates daily.

@msimberg msimberg marked this pull request as ready for review July 25, 2024 09:24
@msimberg
Copy link
Collaborator Author

@bcumming I don't know if I messed up the workflow configuration files between testing and opening the PR. They ran on my fork so you may need to enable something in the settings to allow builds from forks or similar? Or add me to the maintainers?

@msimberg msimberg requested a review from bcumming July 25, 2024 09:30
@msimberg msimberg mentioned this pull request Jul 25, 2024
Merged
bcumming
bcumming requested changes Jul 25, 2024
View reviewed changes
.github/workflows/build_and_test.yml Outdated Show resolved Hide resolved
.github/workflows/clang_format.yml Outdated Show resolved Hide resolved
src/cli/start.cpp Show resolved Hide resolved
src/uenv/lex.cpp Show resolved Hide resolved
src/util/expected.h Show resolved Hide resolved
@msimberg msimberg requested a review from bcumming July 25, 2024 11:42
@bcumming bcumming merged commit 7523315 into eth-cscs:main Jul 25, 2024
3 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@bcumming bcumming bcumming approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@msimberg @bcumming